treeview: bgo#666685 - Crash when toggling the selected status of a row

The refactoring from e01af5c5c seems to have caused this inadvertently.

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

gtkcolorsel: Do not use deprecated GdkColor api

gtkcellview: Do not use deprecated GdkColor api

gtk/gtkprivate.c: Include stdlib.h

Include stdlib.h for getenv to avoid C4013/implicit declaration of ...
warnings

test/testgtk.c: Fix compilation warning

Fix problems with DND on some X servers

When the X server does not support the shape extension (as some
vnc implementations seem to), our DND code was always seeing
an empty input shape, so drops always missed their target.

http://bugzilla.gnome.org/show_bug.cgi?id=620240

Add symbolic constants for event emission control

Similar to https://bugzilla.gnome.org/show_bug.cgi?id=631413 in GLib,
GTK+ should have symbolic constants for controlling the event handling
instead of a mere boolean value.

The GDK_EVENT_PROPAGATE and GDK_EVENT_STOP constants map to FALSE and
TRUE respectively.

https://bugzilla.gnome.org/show_bug.cgi?id=631414

textdisplay: use g_list_free_full

comboboxtext: Set entry-text-column and id-column props in the constructor

This is cleaner than setting the value in the _new() functions and
makes binding and GtkBuilder instantiation easier. Bug #612396, comment 65.

Bug 667018-gtk/gtkwindow.c: Include deprecated/gtkstyle.h

This is needed for the prototype of gtk_widget_reset_rc_styles, to avoid
C4013/implicit declaration of ... warnings of that function

gtk/gtkspinbutton.c: Remove lingering ';'

...so that C89 compilers will not complain

Updated Hebrew translation.

gtk/Makefile.am: Update VS projects completion

Take the sources in gtk/deprecated/ into account-VS does not like '/'
in file paths.

Update Visual C++ projects

-Update include paths for the GTK+ main projects to be consistent across
 the board
-Update demo projects to add a new demo source file, transparent.c

Update gdk/gdkconfig.h.win32

Apparently the deprecation warning macros are placed into gdkconfig.h
during the configure stage, so put these in the pre-configured
gdkconfig.h.win32 as well, as their definitions are needed for all builds.

Remove an unnecessary warning

Don't warn if there's more than one mnemonic label for a widget,
and skip invisible labels.

Bug 665015

Add a Since tag

gtk_css_provider_to_string is a post-3.0 addition.
Fixes bug 659516

Drop -uninstalled.pc files

This has already been done in GLib, so follow suit here.

XIM: gracefully deal with non-X backends

Patch by Young-Ho Cha, bug 660309.

GtkImContextSimple: fix tentative match handling

When switching to the compact table format, we inadvertedly lost
the 'tentative match' handling which allows to complete longer
compose sequences even if a prefix is a complete sequence.
This was pointed out in bug 666710.

Cosmetic changes to comments

Updated POTFILES.skip

settings: Cache style properties

This way we don't need to compute them every lookup. (That's not the
real reason though - the real reason is that I want to add new APIs that
require the caching because they return consts).

modifierstyle: Move Private typedef to public header

reftests: Add test for empty grid with spacing

This used to crash.

https://bugzilla.gnome.org/show_bug.cgi?id=660139

Fix problems with sticky handling

Reading a card32 property into a long may lead to undefined high
bits, so mask them off. Also, make the conditions for setting and
unsetting the stick flag opposites, to avoid unintended changes.
Patch by John Lindgren, bug 666842

Update various EWMH interactions

The current EWMH has added 'source indication' fields to a number
of client messages. Set these to 1 to indicate a regular client.
Also fill the button field of _NET_WM_MOVERESIZE.

tests/testgtk: Do not use GdkColor-based api

Updated Spanish translation

gtktextlayout: use g_slice for GtkTextLineDisplay

gtktextbtree: use g_slice

themingbackground: Avoid erroring cairo context

tests: Make widget-factory close when window is closed

cssprovider: Fix leak

Thanks to the heroic valgrinding efforts of Olivier Sessnik, another
leak could be plugged.

Fix scroll_offset handling

scroll_offset can be negative here, so replace a > 0 test
by != 0.

Only push the menu in if the position function said so

Use gtk_widget_get_allocated_width/height where it makes sense

Remove an unneeded variable

Remove an unused struct

combobox: Don't allow an individual cell to cheat

Make sure we only set sensitive when the cell is sensitive and visible.
Also avoid invalid memory accesses if no cells existed.

widget: Print a g_warning() when a cairo_t is invalid

Only print the warning on internal code, so we aren't too verbose.

styleproperties: Move private struct declaration into public header

x11: Don't handle focus events for NULL windows

The check was accidentally removed in the refactoring of
2ea328dfbc820c32a6f425805e632cd1aef45258.

Revert "x11: Avoid spurious focus events on grabs"

This reverts commit db4a6040afd3d1cabbb306037e811f37d62e2e76.

This is causing bad focus confusion e.g. in gnome-terminal.

Deprecate all the public API that is using GdkColor struct

gtkappchooserwidget: Use g_object_clear()

GtkModelMenu: drop extra & added by mistake

application: don't unconditionally unref the GMenuModel in getters

Both the app-menu and menubar models can be NULL, so we shouldn't
blindly unref them, since g_object_unref() doesn't cope with NULL
pointers.

application: mark menu/menubar getters as (transfer none)

Since they do not return a reference to the underlying GMenuModel.

application: use the same variable names in definitions and declarations

gobject-introspection will be confused and mark the methods as not
introspectable otherwise.

GtkAction: Hold a reference to proxy widgets

Previously we expect that a proxy widget holds a reference to the
action via gtk_activatable_do_set_related_action().  However, it
is possible for the widget to still be in a floating state when
it adds itself to the the action.

This is a problem when gtk_action_get_proxies() gets called, because
we return floating objects back to the user.  And language bindings
aren't going to be expecting that.

Fix this by calling ref_sink() and unref().

https://bugzilla.gnome.org/show_bug.cgi?id=657367

GtkApplicationWindow: fix a typo

And add a link to the screenshots.

tests/testcombo.c: Use GdkRGBA instead GdkColor

gtk/gtkcellview.c: Use new GdkRGBA api instead GdkColor one

gtk/gtktrayicon-x11.c: Use new GdkRGBA api instead GDKColor one

button: deprecate the inner-border style property

Since we allocate the standard CSS border to the button now, and center
the child accordingly, there's no need for an additional inner-border
style property. Deprecate it and ignore its values.

https://bugzilla.gnome.org/show_bug.cgi?id=666600

button: add CSS borders to the size request

And allocate the button children according to it as well.

https://bugzilla.gnome.org/show_bug.cgi?id=666600

button: add a border param to gtk_button_get_props()

We'll use it later to compute the child offsets.

https://bugzilla.gnome.org/show_bug.cgi?id=666600

tests/testgtk.c: Remove death code

gtkmodelmenuitem: don't crash on missing action

Updated Spanish translation

treeview: fix an uninitialised variable warning

Bump version

3.3.6

Fix dist

Fix symbol lists

Updates

Revert "progressbar: Set state flags on draw"

This reverts commit 6ed591871862ec87ee66fe69dca5c42087a12160 which is no
longer needed as the style context now has the state flags updated when going
into draw.

Add another bloatpad screenshot

gdkwindow: plug memory leak

==23282== 64 bytes in 2 blocks are definitely lost in loss record 8,069 of 13,389
==23282==    at 0x4A074CD: malloc (vg_replace_malloc.c:236)
==23282==    by 0x39A1C3E2EA: cairo_region_create (cairo-region.c:196)
==23282==    by 0x6D9AF3D: recompute_visible_regions_internal (gdkwindow.c:964)
==23282==    by 0x6D9B4B8: recompute_visible_regions (gdkwindow.c:1126)
==23282==    by 0x6DA3450: gdk_window_hide (gdkwindow.c:5689)
==23282==    by 0x6D9CED9: _gdk_window_destroy_hierarchy (gdkwindow.c:2042)
==23282==    by 0x6D9D040: gdk_window_destroy (gdkwindow.c:2109)
==23282==    by 0x655B5E4: gtk_entry_unrealize (gtkentry.c:3012)
==23282==    by 0x7068BF3: g_cclosure_marshal_VOID__VOID (gmarshal.c:85)
==23282==    by 0x706710B: g_type_class_meta_marshal (gclosure.c:885)
==23282==    by 0x7066DF9: g_closure_invoke (gclosure.c:774)
==23282==    by 0x7080585: signal_emit_unlocked_R (gsignal.c:3340)
==23282==    by 0x707F619: g_signal_emit_valist (gsignal.c:3033)
==23282==    by 0x707FB71: g_signal_emit (gsignal.c:3090)
==23282==    by 0x679E243: gtk_widget_unrealize (gtkwidget.c:4458)
==23282==    by 0x64E83C7: gtk_bin_forall (gtkbin.c:172)
==23282==    by 0x6548BBD: gtk_container_forall (gtkcontainer.c:2014)
==23282==    by 0x67A966D: gtk_widget_real_unrealize (gtkwidget.c:10253)
==23282==    by 0x672D002: gtk_tool_item_unrealize (gtktoolitem.c:474)
==23282==    by 0x7068BF3: g_cclosure_marshal_VOID__VOID (gmarshal.c:85)

https://bugzilla.gnome.org/show_bug.cgi?id=666552

Updates

https://bugzilla.gnome.org/show_bug.cgi?id=666552

notebook: Use the widget state flags as a base for drawing

menuitem: Stop setting state flags on the style context for drawing

menubar: Stop setting state flags on the style context for drawing

label: Use the widget state flags as a base for drawing

label: Stop setting state flags on the style context for drawing

image: Stop setting state flags on the style context for drawing

iconview: Use the widget state flags as a base for drawing items

iconview: Stop setting state flags on the style context for drawing

hsv: Stop setting state flags on the style context for drawing

expander: Use the widget state flags as a base for drawing

eventbox: Stop setting state flags on the style context for drawing

entry: Use the widget state flags as a base for drawing icons and text

entry: Stop setting state flags on the style context for drawing

combobox: Stop setting state flags on the style context for drawing

cellview: Stop checking for sensitivity on draw

This is needless since GtkCellRenderer does it consistently before rendering.

cellrenderertoggle: Use the cell renderer's state flags as a base for rendering

This allows us to pick the underlying widget's state flags that we don't
explicitly set here.

cellrenderer: Use the widget state flags as a base for cell renderers' state

checkmenuitem: Clean up state flags handling on drawing

checkbutton: Use the widget state flags as a base for drawing the indicator

Instead of building a set of state flags specifically for drawing the
indicator, base it on the underlying widget state flags.

checkbutton: Stop setting state flags on the style context for drawing

calendar: Use the widget state flags as a base for drawing arrows

This allows drawing calendar arrows in all possible states the main widget may
be in.

The arrow_state array is converted into a bit field since it only really needs
to store boolean information about prelight for each arrow.

calendar: Use the widget state flags as a base for drawing days

Instead of building a set of state flags specifically for drawing days, base
it on the underlying widget state flags.

button: Stop setting state flags on the style context for drawing

arrow: Stop setting state flags on the style context for drawing

accellabel: Stop setting state flags on the style context for drawing

widget: Set style flags on the style context when the flags are set

This will allow us to simplify most draw methods by not having to explicitly
set the style flags on the context.

widget: Always propagate state flags to all children

https://bugzilla.gnome.org/show_bug.cgi?id=666126

Updated POTFILES.in

bump glib depend to 2.31.6